-
Notifications
You must be signed in to change notification settings - Fork 10
Chore/ci workflows #193
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Chore/ci workflows #193
Conversation
SeddikBellamine
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good work, overall we need to make some changes left you some comments to reconsider
| @@ -0,0 +1,10 @@ | |||
| { | |||
| "packages": { | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
for some reason the actual version of release please dosn't support multiple packages, and it goes like this to make it work:
{ ".": "1.2.2" }
| readFileSync('../dapp/package.json', 'utf-8') | ||
| ).version; | ||
|
|
||
| export const DOCKER_IMAGE_NAMESPACE = 'iexechub'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we won't need the SCONIFIER_VERSION and dappVersion as we are using input tag from env variable
| "changelog-path": "CHANGELOG.md", | ||
| "include-v-in-tag": true | ||
| }, | ||
| "dapp": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
multiple packages dosn't work we need to investigate plus
.github/workflows/dapp-deploy.yml
Outdated
| env: | ||
| DEPLOY_ENVIRONMENT: ${{ inputs.environment }} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can't find where this env is consumed
.github/workflows/dapp-deploy.yml
Outdated
| DOCKER_IMAGE_DEV_TAG: ${{ env.DOCKER_IMAGE_DEV_TAG }} | ||
| DOCKER_IMAGE_PROD_TAG: ${{ env.DOCKER_IMAGE_PROD_TAG }} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
these two seems to be empty
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we may want to refactor using enviroments secrets instead of dev/prod in en names https://docs.github.com/en/actions/how-tos/security-for-github-actions/security-guides/using-secrets-in-github-actions#creating-secrets-for-an-environment
c28d528 to
aa2733e
Compare
- Remove .drone.yml configuration - Add GitHub Actions workflows for CI/CD - Update .gitignore for GitHub Actions artifacts - Add workflows for SDK and Dapp CI/CD pipelines
- Add release-please configuration for automated versioning - Add release manifest for tracking version changes - Enable conventional commit checks for PR titles and commits
- Migrate deployment scripts from Drone to GitHub Actions environment variables - Update configuration to support multi-environment deployments - Improve environment-specific whitelisted apps configuration - Update deployment scripts for new workflow compatibility
- Update test utilities for new environment configuration - Update environment preparation scripts for multi-env support - Update dapp e2e tests for new environment configuration - Improve test environment setup for CI/CD workflows
08de737 to
e9d4cf8
Compare
8e0a683 to
69bcd03
Compare
…order steps and use only dev tag
SeddikBellamine
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
No description provided.